batch/helper/timeformat.test.ts (1)

5-15: 旧フォーマットの回帰テストも追加したいです。

移行期間は YYYY-MM-DD HH:mm:ss 形式の既存データが残り得るので、それも UTC として解釈できることをここで固定しておくと安心です。

💡 追加イメージ

 describe('timeFormatTz', () => {
   it('converts ISO 8601 UTC to Asia/Tokyo', () => {
     expect(timeFormatTz('2026-03-16T02:56:35Z', 'Asia/Tokyo')).toBe(
       '2026-03-16 11:56:35',
     )
   })
+
+  it('treats legacy UTC strings without Z as UTC', () => {
+    expect(timeFormatTz('2026-03-16 02:56:35', 'Asia/Tokyo')).toBe(
+      '2026-03-16 11:56:35',
+    )
+  })

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@batch/helper/timeformat.test.ts` around lines 5 - 15, 既存のISOテストに加えて旧フォーマット
'YYYY-MM-DD HH:mm:ss' を UTC として解釈する回帰テストを追加してください:
batch/helper/timeformat.test.ts にある timeFormatTz を使い、例えば
timeFormatTz('2026-03-16 02:56:35', 'Asia/Tokyo') が '2026-03-16 11:56:35'
を返すケースと timeFormatTz('2026-03-16 02:56:35', 'US/Pacific') が '2026-03-15
19:56:35' を返すケースを追加して、旧フォーマットを明示的に UTC として扱うことを固定化してください.

batch/github/review-response.test.ts (1)

50-56: 固定日付によるテストは意図が伝わりにくい可能性があります。

2025-01-01という固定日付では「90日より古い」というテストの意図がコードから読み取りにくいです。相対日付を使用すると、テストの意図が明確になり、将来の保守性が向上します。

♻️ 相対日付を使用した改善案

+import dayjs from 'dayjs'
+import utc from 'dayjs/plugin/utc'
+
+dayjs.extend(utc)
+
 // ...
 
   it('filters out comments older than 90 days', () => {
+    const oldDate = dayjs.utc().subtract(91, 'days').toISOString()
     const result = analyzeReviewResponse([
-      comment('alice', '2025-01-01T10:00:00Z'),
-      comment('bob', '2025-01-01T12:00:00Z'),
+      comment('alice', oldDate),
+      comment('bob', dayjs.utc(oldDate).add(2, 'hours').toISOString()),
     ])
     expect(result).toEqual([])
   })

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@batch/github/review-response.test.ts` around lines 50 - 56, The test using a
fixed date makes the "older than 90 days" intent unclear; update the test around
analyzeReviewResponse and the test helper comment(...) to use relative dates
(e.g., compute Date.now() or mock current time and create timestamps like now
minus 91 days and now minus 89 days) so the expectation remains stable over
time; alternatively mock the global Date/Date.now in the test and generate
comment timestamps based on that mocked "now" to clearly express "older than 90
days" vs "within 90 days."